[データベース] 各テンプレートで一時保存や公開前ラベルを初期表示画面（一覧画面）に表示#2382
Open
akagane99 wants to merge 5 commits intoopensource-workshop:masterfrom
Open
[データベース] 各テンプレートで一時保存や公開前ラベルを初期表示画面（一覧画面）に表示#2382akagane99 wants to merge 5 commits intoopensource-workshop:masterfrom
akagane99 wants to merge 5 commits intoopensource-workshop:masterfrom
Conversation
masaton0216
reviewed
Mar 25, 2026
Contributor
masaton0216
left a comment
masaton0216
left a comment
There was a problem hiding this comment.
ご対応ありがとうございます。
スペース除去依頼２件と、意図通りの挙動か確認が２件です。
resources/views/plugins/user/databases/design-table-dl/databases_detail.blade.php
Outdated
Show resolved
Hide resolved
resources/views/plugins/user/databases/card_02/databases_detail.blade.php
Outdated
Show resolved
Hide resolved
| @include('plugins.user.databases.default.databases_include_value') | ||
|
|
||
| {{-- ステータス表示のみ --}} | ||
| @include('plugins.user.databases.default.databases_include_status_and_button', [ |
Contributor
There was a problem hiding this comment.
（意図通りかの確認コメントです）
実用上は大きな問題にはならないのですが、バッジの役目が「ステータス表示」と「詳細ページへのナビリンク」の２役になっています。
正味の役割としては「ステータス表示」だけかと思うので、バッジをクリッカブルな要素にすると「クリックして何かが起きる要素なのか？」とユーザによっては期待と実際の挙動にギャップが生じる可能性があるのかなと思いました。
※実用上は実害がない箇所なので、このまま通すことも可能です。
| @include('plugins.user.databases.default.databases_include_value') | ||
|
|
||
| {{-- ステータス表示のみ --}} | ||
| @include('plugins.user.databases.default.databases_include_status_and_button', [ |
Contributor
There was a problem hiding this comment.
（意図通りかの確認コメントです）
実用上は大きな問題にはならないのですが、バッジの役目が「ステータス表示」と「詳細ページへのナビリンク」の２役になっています。
正味の役割としては「ステータス表示」だけかと思うので、バッジをクリッカブルな要素にすると「クリックして何かが起きる要素なのか？」とユーザによっては期待と実際の挙動にギャップが生じる可能性があるのかなと思いました。
※実用上は実害がない箇所なので、このまま通すことも可能です。
Contributor
Author
|
@masaton0216 さん |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
修正後画面
tableテンプレート 初期表示画面（一覧画面）
design-table-dlテンプレート
初期表示画面（一覧画面）
詳細画面
defaultテンプレート
初期表示画面（一覧画面）
詳細画面
card_02テンプレート
初期表示画面（一覧画面）
詳細画面
レビュー完了希望日
急ぎません
関連Pull requests/Issues
参考
なし
DB変更の有無
なし
チェックリスト